-
Notifications
You must be signed in to change notification settings - Fork 1.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(pictograms): add four new pictograms to education category #6353
feat(pictograms): add four new pictograms to education category #6353
Conversation
Deploy preview for carbon-elements ready! Built with commit 311d134 |
Deploy preview for carbon-components-react ready! Built with commit 311d134 https://deploy-preview-6353--carbon-components-react.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
updated snapshots, looks good to me once CI passes
f0263de
to
6a37184
Compare
@tw15egan @joshblack after I created this PR, I was asked to change the name of one of the SVGs. The previously submitted |
Hey @dudley-ibm! 👋 Feel free to rename focus.svg to whatever the preferred name would be! Since you are adding this in this PR, and it's not in the library yet, there's also no need to worry about the deprecation flow and you can just make the changes directly. |
@dudley-ibm I've goen ahead and updated
Let me know if this is what you had intended to change 👍 |
Thanks @tw15egan! Here's what I was trying to accomplish:
|
b9e84f7
to
311d134
Compare
Closes #
{{closes enterprise issue #6 add 4 pictograms}}
Changelog
New
goals.svg
growth.svg
insights.svg}}